Skip to content

Conversation

@arboleya
Copy link
Contributor

@arboleya arboleya commented Dec 16, 2024

Checklist

  • All changes are covered by tests (or not applicable)
  • All changes are documented (or not applicable)
  • I reviewed the entire PR myself (preferably, on GH UI)
  • I described all Breaking Changes (or there's none)

@vercel
Copy link

vercel bot commented Dec 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fuels-template ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 31, 2024 9:38am
ts-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 31, 2024 9:38am
ts-docs-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 31, 2024 9:38am

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 16, 2024

CodSpeed Performance Report

Merging #3486 will degrade performances by 20.44%

Comparing aa/fix/ui-script-call (6793369) with master (41c72fb)

Summary

❌ 1 regressions
✅ 17 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark master aa/fix/ui-script-call Change
should successfully conduct a custom transfer between wallets (x20 times) 50.3 ms 63.2 ms -20.44%

@petertonysmith94
Copy link
Contributor

Are these tests being run in the CI?

@arboleya
Copy link
Contributor Author

Are these tests being run in the CI?

I don't think so, but will defer to @Dhaiwat10 @danielbate.

@petertonysmith94 What is your concern?

@petertonysmith94
Copy link
Contributor

I don't think so, but will defer to @Dhaiwat10 @danielbate.

@petertonysmith94 What is your concern?

The main concern would be shipping broken tests with the template.

@github-actions
Copy link
Contributor

Coverage Report:

Lines Branches Functions Statements
77.83%(+0%) 70.5%(+0.03%) 75.5%(+0%) 77.79%(+0%)
Changed Files:
Ok File (✨=New File) Lines Branches Functions Statements
🔴 packages/account/src/providers/transaction-request/transaction-request.ts 88.57%
(+0%)
78.08%
(+1.37%)
84%
(+0%)
88.81%
(+0%)

@arboleya arboleya merged commit 4c5da1d into master Jan 2, 2025
23 checks passed
@arboleya arboleya deleted the aa/fix/ui-script-call branch January 2, 2025 08:47
@arboleya arboleya changed the title fix: test ui script in package's json fix: adjust test ui script in package's json Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue is a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't run test:ui inside new create fuels app

6 participants